-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update template for docs issues #10731
Conversation
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @nghi-ly |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10731 +/- ##
==========================================
+ Coverage 88.90% 88.95% +0.05%
==========================================
Files 180 181 +1
Lines 22856 22959 +103
==========================================
+ Hits 20319 20424 +105
+ Misses 2537 2535 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @nghi-ly |
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @nghi-ly |
hmm, i did sign and submit the CLA. not sure why i'm still getting this error #10731 (comment) |
.github/workflows/docs-issue.yml
Outdated
@@ -37,5 +37,6 @@ jobs: | |||
with: | |||
issue_repository: "dbt-labs/docs.getdbt.com" | |||
issue_title: "[Core] Docs Changes Needed from ${{ github.event.repository.name }} Issue #${{ github.event.issue.number }}" | |||
issue_body: "At a minimum, update body to include a link to the page on docs.getdbt.com requiring updates and what part(s) of the page you would like to see updated." | |||
issue_body: "At a minimum, update body to include a link to the page on docs.getdbt.com requiring updates and what part(s) of the page you would like to see updated.\n Originating from this issue: https://github.com/dbt-labs/dbt-core/issues/${{ github.event.issue.number }}" | |||
label: 'dbt-core v1.9' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we're opening an issue in another repo we can't actually add any labels. The community-label.yml
workflow works entirely within the core repo which is why it can add labels to issues.
Right now you're adding labels to these autogenerated issues within your own repo with autogenerated_labeler.yml.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i see. thank you for the info and extra context
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @nghi-ly |
@cla-bot check |
The cla-bot has been summoned, and re-checked this pull request! |
Description
Update template for auto-creating Docs Issues to include:
user_docs
labelAdd labeldbt-core v1.9
-- this should be updated / incremented at the end/start of the next versionChecklist